Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Vue 3 Support #99

Merged
merged 10 commits into from
Oct 23, 2022
Merged

Conversation

fech-dev
Copy link
Contributor

This PR adds the plugin's Vue 3 support + useShepherd() composable function for the usage in setup hooks.

To make that possible I use vue-demi library, but there are some limitations in terms of building the package.

In package.json's module field I have changed it to src/entry.js because vue-demi works at build time, so I remove the ESM compilation since it was unnecessary and user can directly use src/entry.js.

The problem is for the SSR and unpkg versions... I think that the solution can be to build two versions of each build type, where:
One will still support Vue 2 (files named vue2-shepherd.min.js & vue2-shepherd.ssr.js)
and the others for Vue 3 (files named vue3-shepherd.min.js & vue3.shepherd.ssr.js).

These changes are not in this commit, if you are ok with this solution I'll make the change.

@fech-dev fech-dev marked this pull request as draft July 23, 2022 14:56
@fech-dev fech-dev marked this pull request as ready for review July 23, 2022 15:30
@fech-dev fech-dev mentioned this pull request Jul 23, 2022
@RobbieTheWagner
Copy link
Member

@fech-dev thank you so much for the PR and sorry for the delay in responding! After thinking about it some more, I think I am okay with completely dropping Vue 2 support, so if you would like to proceed with updating to entirely Vue 3 and composables, I support that 👍

@fech-dev
Copy link
Contributor Author

fech-dev commented Sep 2, 2022

@fech-dev thank you so much for the PR and sorry for the delay in responding! After thinking about it some more, I think I am okay with completely dropping Vue 2 support, so if you would like to proceed with updating to entirely Vue 3 and composables, I support that 👍

Thanks @rwwagner90. Ok, I'll try to update asap.

@RobbieTheWagner
Copy link
Member

@fech-dev did you still want to update this PR?

@fech-dev
Copy link
Contributor Author

@fech-dev did you still want to update this PR?

Sorry, for the late reply, but I was very busy.
I've made a commit, let me know if it's ok.

@RobbieTheWagner
Copy link
Member

@fech-dev do you mind rebasing this please?

@RobbieTheWagner RobbieTheWagner merged commit 6c62d30 into shipshapecode:master Oct 23, 2022
@RobbieTheWagner
Copy link
Member

@fech-dev thanks for all the hard work! Just released 2.0.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants